Skip to content

Floran Speech Revert #6453

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
May 2, 2025
Merged

Floran Speech Revert #6453

merged 12 commits into from
May 2, 2025

Conversation

Evalence0
Copy link
Contributor

@Evalence0 Evalence0 commented Apr 23, 2025

About The Pull Request

This pull request should bring Floran speech back in line with how lizard speech works, primarily re-implementing hissing. I am not a coder, but this bugged me for a little while and felt too small to list as an issue. I figured I would give it a shot myself, so hopefully it works.

Why It's Good For The Game

In their source material, Starbound, florans have trouble saying the "s" sound, drawing it out heavily. It's one of the fun quirks of their speech, so I was a bit disappointed when suddenly it was gone from monke and replaced with the speech prefix "whistles," which just doesn't really fit the savage plant people all that well.

Changelog

🆑
changed: floran speech from "whistles" to "hisses."
code: copied draw_length and other logic from lizard tongues to floran tongues
/:cl:

Attempting to revert the Floran speech changes that happened a while ago (around when Ornithids got added)
@Evalence0
Copy link
Contributor Author

Evalence0 commented Apr 23, 2025

You know, I really hope this isn't a stupid PR, I never really have used github. I only know basic coding stuff, so I'm SURE there will be something wrong with how I did this, but hopefully my idea gets across and it can be properly implemented by someone that knows what they're doing!

Also, not sure if this will implement the hiss length variable like is available on character creation for lizards.

@Kitsunemitsu Kitsunemitsu added the Tweak change a thing to work a little different label Apr 23, 2025
@flleeppyy
Copy link
Member

Personally, I would say that the speech modification proc be it's own proc to be used across the codebase, but the polyglot tongue has most of it's conversion/modification stuff in one big proc too, so I think this is fine until someone turns each of the speech modifications in to their own global proc.

@flleeppyy flleeppyy added the Approval: awaiting admin review PR is awaiting admin approval label Apr 25, 2025
@notghosti
Copy link
Collaborator

image

@Evalence0
Copy link
Contributor Author

I think I fixed the oopsie, there was no client side way to set hiss length so it depended on random map generation. Now it should hopefully pop up during character creation for florans, so you can set just how much you want to hiss.

@Evalence0
Copy link
Contributor Author

I am very, very sorry that I am not good at this. This time it should be working, hopefully.

@Veth-s Veth-s added Approval: admin approved PR is approved by admins and removed Approval: awaiting admin review PR is awaiting admin approval labels Apr 30, 2025
@Absolucy Absolucy merged commit 73168e4 into Monkestation:master May 2, 2025
25 checks passed
github-actions bot added a commit that referenced this pull request May 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approval: admin approved PR is approved by admins Tweak change a thing to work a little different
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants